Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: Fix Flake8 issues in example files #4353

Merged
merged 8 commits into from
Sep 21, 2024

Conversation

arohanajit
Copy link
Contributor

This PR fixes the flake8 error in doc/python. It addresses mainly F403, F405 and E501. These are mostly related to wild card imports and max-line-length errors. I haven't fixed F401 in dialogs.py yet as that will require more comprehensive changes

@github-actions github-actions bot added Python Related code is in Python docs labels Sep 20, 2024
doc/python/m.distance.py Outdated Show resolved Hide resolved
doc/python/raster_example_ctypes.py Outdated Show resolved Hide resolved
doc/python/vector_example_ctypes.py Outdated Show resolved Hide resolved
arohanajit and others added 3 commits September 20, 2024 15:20
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Member

@wenzeslaus wenzeslaus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These need more work to be easily testable. Let's merge this as is relying on the static checks making sure that the PR does not make the code worse.

@wenzeslaus wenzeslaus changed the title CI: Resolve flake8 issues in doc directory doc: Fix Flake8 issues in example files Sep 21, 2024
@wenzeslaus wenzeslaus merged commit 2356520 into OSGeo:main Sep 21, 2024
27 checks passed
@arohanajit arohanajit deleted the aajit-flake8-doc branch September 21, 2024 08:40
@neteler neteler added this to the 8.5.0 milestone Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Python Related code is in Python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants